fix(web): hide mobile sidebar after thread actions#1259
fix(web): hide mobile sidebar after thread actions#1259chirino wants to merge 1 commit intopingdotgg:mainfrom
Conversation
Close the off-canvas sidebar on narrow screens when users create a new thread or select an existing one. Signed-off-by: Hiram Chirino <hiram@hiramchirino.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can enforce grammar and style rules using `languagetool`.Configure the |
Summary
Hide the off-canvas sidebar on narrow screens after users create a new thread or select an existing one. This keeps the selected thread content visible immediately instead of leaving the mobile navigation open. Fixes #1258.
Changes
Note
Hide mobile sidebar after thread actions in the Sidebar component
On mobile, the sidebar now closes when a user opens a thread (via click or Enter/Space) or creates a new thread from a project card. A new
shouldHideSidebarAfterThreadActionhelper in Sidebar.logic.ts wraps theisMobileboolean to centralize the condition. On desktop, sidebar behavior is unchanged.Macroscope summarized 73de16b.